Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[full-ci] Remove EventDispatcherInterface::dispatch from phpstan ignoreErrors #586

Merged
merged 1 commit into from
May 23, 2023

Conversation

phil-davis
Copy link
Contributor

Fixes #585

Now that Symfony5 is in core, phpstan understands the correct parameter order for EventDispatcherInterface::dispatch
The entry in ignoreErrors is no longer needed.

@phil-davis phil-davis self-assigned this May 23, 2023
@phil-davis phil-davis changed the title Remove EventDispatcherInterface::dispatch from phpstan ignoreErrors [full-ci] Remove EventDispatcherInterface::dispatch from phpstan ignoreErrors May 23, 2023
@phil-davis phil-davis force-pushed the adjust-for-symfony5 branch from 7594451 to 39c0235 Compare May 23, 2023 05:23
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@phil-davis phil-davis requested review from SwikritiT and saw-jan May 23, 2023 06:30
@phil-davis phil-davis merged commit 3149244 into master May 23, 2023
@delete-merged-branch delete-merged-branch bot deleted the adjust-for-symfony5 branch May 23, 2023 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Nightly is failing in phpstan
2 participants